Skip to content

chore(github): add CODEOWNERS#1359

Merged
jaredvu merged 2 commits intomainfrom
security-audit-codeowners
Feb 11, 2026
Merged

chore(github): add CODEOWNERS#1359
jaredvu merged 2 commits intomainfrom
security-audit-codeowners

Conversation

@UnbornAztecKing
Copy link
Contributor

Summary

  • Introduces CODEOWNERS to formalize default code ownership.
  • Sets repo-wide default owners to @dydxprotocol/web-code-owners.
  • Assigns @dydxprotocol/security ownership for .github and CODEOWNERS to enforce governance on security/policy files.

Details

  • Repo governance:
    • New CODEOWNERS file at repo root.
    • Default owner: @dydxprotocol/web-code-owners for all paths.
    • Security team is the sole owner for:
      • /.github/ (workflows, issue templates, policies)
      • /CODEOWNERS (prevents unilateral changes to ownership rules)

Risk & Impact

  • Low risk: affects review workflow only.
  • If branch protection enforces CODEOWNERS, PRs touching .github/ or CODEOWNERS will require approval from @dydxprotocol/security.
  • No runtime, API, or build changes.

Testing

  • No tests added; governance configuration is exercised by GitHub’s CODEOWNERS enforcement.

Reviewer Notes

  • Verify branch protection rules enforce CODEOWNERS as intended.
  • Confirm the listed teams exist and have appropriate permissions.

@vercel
Copy link

vercel bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v4-localization Ready Ready Preview, Comment Feb 11, 2026 2:08am

Request Review

jaredvu
jaredvu previously approved these changes Feb 10, 2026
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@UnbornAztecKing copilot is saying that dydxprotocol/web-code-owners does not exist. (I think it is @dydxprotocol/web-code-owner . Should we update to owners instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@jaredvu jaredvu merged commit 73a18c1 into main Feb 11, 2026
6 checks passed
@jaredvu jaredvu deleted the security-audit-codeowners branch February 11, 2026 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants